Feat/missing pages audit implementation#161
Merged
0xdevcollins merged 8 commits intoJun 25, 2026
Merged
Conversation
- Updated package.json build script to copy necessary Prisma package.json files. - Modified schema.prisma to include Notification model and related database migrations. - Implemented NotificationsModule and NotificationsService for handling notifications. - Integrated NotificationsController for API endpoints to list and manage notifications. - Created NotificationBell component for displaying notifications in the dashboard. - Added RealtimeToastNotifications provider for real-time toast notifications based on socket events. - Introduced useToastNotificationPreference hook for managing user preferences on toast notifications. - Updated EventsService to emit notification events when relevant actions occur. - Enhanced AuthService and InvoicesService to trigger notifications on specific actions.
…/Refund-Management-UI
…sections - Updated TypeScript configuration for better readability. - Created new blog post pages with dynamic routing and metadata generation. - Added a changelog RSS feed for blog updates. - Implemented a contact page with team contact information. - Developed an integrations page showcasing various integration options. - Introduced a press release page with dynamic content based on slug. - Added product pages with detailed descriptions and CTAs. - Created a security researchers recognition page. - Established data structures for blog posts and product pages. - Included placeholder assets for branding and marketing materials.
…/Missing-Pages-Audit-Implementation
|
@Ekene001 is attempting to deploy a commit to the Threadflow Team on Vercel. A member of the Team first needs to authorize it. |
|
@Ekene001 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
Owner
|
@Ekene001 Please resolve conflict |
Resolved 13 conflicts: - payouts.module.ts / payouts.service.ts: keep NotificationsService injection (HEAD) - (auth)/layout.tsx, layout.tsx, globals.css: keep main's dark mode tokens and flash-prevention script - links/page.tsx, blog/*, products/page.tsx, Footer.tsx: keep main's canonical implementations - site-header.tsx: merge — NotificationBell (HEAD feature) + ThemeToggle (main's component) - useDashboardSocket.ts: keep HEAD's robust singleton with useAuth and reconnection logic - sitemap.ts: merge — main's getAllPosts import, HEAD's extra routes (staticRoutes, customerStories, pressReleases)
…Implementation - schema.prisma: keep both notifications (HEAD) and recipients (main) relations on Merchant - payouts.service.ts: auto-merged cleanly
- Remove invalid DTO.schema property assignments (types are erased at runtime) - Export schemas directly and reference them by name in controller and update DTO - Cast recipient.details through unknown to match CreatePayoutDto destination union type
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
closes #143
PR Title
Marketing Site Audit Completion: Implement Missing Routes, Product and Blog IA, and Sitemap Coverage
Summary
This PR implements the missing marketing-site pages identified in the audit and closes route gaps causing broken user journeys and 404s.
The update delivers complete coverage for primary navigation, footer links, homepage CTA destinations, sitemap entries, and dynamic content routes so referenced marketing pages resolve successfully.
What’s Included
Priority 0 (Critical)
Priority 1 (Product Pages)
Added four product detail pages:
Invoicing
Global Payouts
Priority 2 (Marketing, Legal, Compliance)
To satisfy the requirement that all referenced routes resolve:
Expanded sitemap coverage to include:
Legal/compliance/security routes
This improves crawlability, discoverability, and procurement-readiness expectations for a B2B SaaS site.
UX/Business Impact
Validation